Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v15] Add pyroscope support #51480

Merged
merged 2 commits into from
Jan 27, 2025
Merged

Conversation

evanfreed
Copy link
Contributor

@evanfreed evanfreed commented Jan 24, 2025

Backport #51294 to branch/v15

Had a few merge conflicts with go.mod / sum let me know if these don't look right.

changelog: Add support for continuous profile collection with Pyroscope

* feat: add pyroscope sdk support

Signed-off-by: Evan Freed <evan.freed@goteleport.com>

* Update lib/service/pyroscope.go

Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com>

* Update lib/service/pyroscope.go

Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com>

* Update lib/service/service.go

Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com>

* review comments

Signed-off-by: Evan Freed <evan.freed@goteleport.com>

* license fix

Signed-off-by: Evan Freed <evan.freed@goteleport.com>

* Update lib/service/pyroscope.go

Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com>

* fmt

Signed-off-by: Evan Freed <evan.freed@goteleport.com>

---------

Signed-off-by: Evan Freed <evan.freed@goteleport.com>
Co-authored-by: rosstimothy <39066650+rosstimothy@users.noreply.github.com>
Signed-off-by: Evan Freed <evan.freed@goteleport.com>
@evanfreed evanfreed added this pull request to the merge queue Jan 27, 2025
Merged via the queue into branch/v15 with commit ef3ef7d Jan 27, 2025
36 checks passed
@evanfreed evanfreed deleted the evan/backport-51294-branch/v15 branch January 27, 2025 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants